Skip to content

fix(cli): resolve OpenAPI references and validate Linux installs - #190

Merged
nkavian merged 1 commit into
inflowpayai:mainfrom
nkavian:fix/openapi-local-schema-references
Nov 1, 2026
Merged

nkavian merged 1 commit into
inflowpayai:mainfrom
nkavian:fix/openapi-local-schema-references

Conversation

@nkavian

@nkavian nkavian commented Nov 1, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Resolve same-document OpenAPI schema references in operation parameters and JSON request bodies, including nested objects and arrays. Unsupported references remain explicit preparation errors without additional schema downloads.
  • Support Arch Linux package installation, removal, and purge through the hosted installer.
  • Check vault service readiness after RPM installation in both the hosted and versioned Linux installers. A package-manager success no longer hides vault startup failure.
  • Give MCP smoke-test setup a separate bounded deadline so vault reset/reinitialization does not consume the request-response deadline; report the phase when a timeout occurs.

Validation

  • pnpm verify passed: 3,596 package tests passed, two skipped; changed-line coverage 97.37%.
  • The four macOS reset/logout cases and a controlled 11-second setup regression passed against the built CLI.
  • AgentMail's public OpenAPI document successfully prepared the previously failing domain request without invoking the operation.
  • Corrected hosted installer installed signed 0.15.0 on Arch x86-64; MCP doctor passed with 55 tools.
  • Real Fedora 42 ARM64 tests reproduced the original false-success result. Both corrected installers exit unsuccessfully when namespace restrictions prevent vault startup and succeed when the required isolation is available. MCP doctor passed with 55 tools and vault status responded successfully.
  • Shell syntax and whitespace checks passed.

The OpenAPI resolver intentionally leaves external schema references, reference siblings, changed schema scopes, and unresolved cycles unsupported. The installer reports unsupported runtime isolation rather than bypassing it.

@codecov

codecov Bot commented Nov 1, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 97.36842% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
packages/core/src/openapi/reader.ts 97.36% 1 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@nkavian
nkavian force-pushed the fix/openapi-local-schema-references branch from f23ed7b to 6100547 Compare November 1, 2026 02:59
@nkavian
nkavian merged commit eeca515 into inflowpayai:main Nov 1, 2026
6 checks passed
@nkavian
nkavian deleted the fix/openapi-local-schema-references branch November 1, 2026 03:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant